Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

First demo video #58

Merged
merged 5 commits into from
Apr 2, 2024
Merged

First demo video #58

merged 5 commits into from
Apr 2, 2024

Conversation

clement-moulin-frier
Copy link
Owner

Description

@corentinlger @Marsolo1 Please check and see if you could make the readme more appealing with vid or gif

@corentinlger
Copy link
Collaborator

I'm not a huge fan of the Readme render at the moment, but did you imagine something like that @clement-moulin-frier ?

@clement-moulin-frier
Copy link
Owner Author

The ideal would be to have a gif of the entire screen of the youtube video, starting at 8s. Is it possible or is it too long for a GIF? If too long, please let me know what is the maximum duration we could have. Thanks

@Marsolo1
Copy link
Collaborator

I think that a gif shouldn't last more than a few seconds. Maybe we could have the same gif as the current one but with the whole screen instead of just the plot, and a bigger link to the video below?

@clement-moulin-frier
Copy link
Owner Author

Sounds good to me, let's have a short gif where we see the interaction with the interface, e.g. from 1:00 to 1:15 in the full vid (or recording a new short vid just for this purpose). Making the link to the video more visible is a good idea

@Marsolo1
Copy link
Collaborator

Marsolo1 commented Mar 27, 2024

I made a gif with the full screen, including config modifications and a start so we see the entities moving (also made the video link bigger)

@Marsolo1
Copy link
Collaborator

@clement-moulin-frier you can tell me what you think, if I should change it or not

@clement-moulin-frier
Copy link
Owner Author

It works for me @Marsolo1 , you can merge it. Thanks

@Marsolo1 Marsolo1 merged commit d0a77eb into main Apr 2, 2024
2 checks passed
@corentinlger corentinlger deleted the clement/demo-video branch April 4, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants